Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update and Add Binder #13

Merged
merged 2 commits into from Dec 26, 2017
Merged

Update and Add Binder #13

merged 2 commits into from Dec 26, 2017

Conversation

radovankavicky
Copy link
Contributor

For those who don't want to install and use Docker (or avoid it).

Similarly like Jake has it here: https://github.com/jakevdp/PythonDataScienceHandbook

Binder simply builds a Docker image of your repository & will search for a dependency file, such as requirements.txt or environment.yml, in the repository's root directory and render all the notebooks for you.

+Maybe an index file could be added in future... but I will leave that for you (if you want it).

For those who don't want to install and use Docker.
Here is a list of dependencies (you can modify it according to the notebooks and libraries there added)
@radovankavicky
Copy link
Contributor Author

Added also list of dependencies (you can modify it according to the notebooks and libraries there added)

@radovankavicky
Copy link
Contributor Author

@sachinruk sachinruk merged commit 59cf2eb into sachinruk:master Dec 26, 2017
@sachinruk
Copy link
Owner

@radovankavicky Im actually getting the following error when I try to launch it. Any thoughts?

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:59cf2ebc3fa166accc630e42a5e7af59ea334
673 for user sachinruk-deepschool.io-uicpr1kp took too long to launch

@radovankavicky
Copy link
Contributor Author

radovankavicky commented Jan 20, 2018

I tried it again too & now taking a long time to load and nothing. After I clicked to see the logs for details (as suggested by Binder) and there was only:

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:d3b02a5da91a2de56e3d0c39f8363d2a3b3fd137 for user sachinruk-deepschool.io-84v8w49c took too long to launch

Similar issue is described here: binder-project/binder#88

Maybe it is because of changes in repo: binder-project/binder#41

Or maybe we need to update requirements.txt document within the repo (to update list of dependencies).

Not sure how to resolve it (it was working only at the beginning/not after the changes in repository).

I wrote & described the issue on their Gitter (hopefully there will be somebody from the Binder developers who could help/point in the right direction how to resolve the issue).

@radovankavicky
Copy link
Contributor Author

@sachinruk
So, here is how to resolve it (reaction directly from Binder developers/they are amazing... reacted very quickly, so let's try to do it the way they suggested):

Tim Head @betatim Jan 20 22:29
@radovankavicky you have a Dockerfile in your deepschool.io repo, so binder is trying to build that image and expects it to start a notebook server in a particular way
if you aren't using the dockerfile to build the binder image then I think the best way to get binder to ignore it is to create a directory called binder/ and place a requirements.txt in it that contains your dependencies
i think as soon as there is a binder/ subdirectory we ignore everything in the top level directory

Chris Holdgraf @choldgraf Jan 20 23:07
@radovankavicky I made a PR to do this, see my fork binder link here: https://mybinder.org/v2/gh/choldgraf/deepschool.io/binder_folder?filepath=notebooks%2FIndex.ipynb
however, I noticed you've got a bug in your requirements code (not your bug, but a scikit-image bug)
the last version of skimage depends on a version of numpy that has a bug in it, which means you won't be able to pip install -r with that skimage version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants